Skip to content

Comments

Added Change to parse Vlan Config without any VlanInterface binded to it#2000

Open
abdosi wants to merge 1 commit intoAzure:202412from
abdosi:202412
Open

Added Change to parse Vlan Config without any VlanInterface binded to it#2000
abdosi wants to merge 1 commit intoAzure:202412from
abdosi:202412

Conversation

@abdosi
Copy link
Contributor

@abdosi abdosi commented Feb 18, 2026

What I did:

Added Change to parse Vlan Config without any VlanInterface binded to it

How I verify:

Before fix:

ABDOSI@xxxx-0101-0040-12xx~$ sonic-cfggen -m minigraph.xml -v DEVICE_METADATA
Traceback (most recent call last):
  File "/usr/local/bin/sonic-cfggen", line 510, in <module>
    main()
  File "/usr/local/bin/sonic-cfggen", line 393, in main
    deep_update(data, parse_xml(minigraph, platform, asic_name=asic_name))
                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/dist-packages/minigraph.py", line 2097, in parse_xml
    (intfs, lo_intfs, mvrf, mgmt_intf, voq_inband_intfs, vlans, vlan_members, dhcp_relay_table, pcs, pc_members, acls, acl_table_types, vni, tunnel_intfs, dpg_ecmp_content, static_routes, tunnel_intfs_qos_remap_config) = parse_dpg(child, hostname)
                                                                                                                                                                                                                             ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/dist-packages/minigraph.py", line 1045, in parse_dpg
    vmbr_list = vintfmbr.split(';')

After fix:

Parsing is fine

Signed-off-by: Abhishek Dosi <abdosi@microsoft.com>
@r12f
Copy link

r12f commented Feb 19, 2026

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@abdosi
Copy link
Contributor Author

abdosi commented Feb 23, 2026

@r12f : can you help with merge of this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants